feat(appkit): split the agents plugin into focused modules (#532) - #547

…ader

Step 2 of splitting the agents plugin. Moves skill discovery, per-agent
catalog resolution, and the load_skill/read_skill_file dispatch into
skill-loader.ts as free functions; the class keeps thin delegators (call sites
unchanged) and skillWorkspaceClient() as the OBO credential seam.

agents.ts: 2331 -> 2152 lines. typecheck + 394 agent tests green.

Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>

Step 3 of splitting the agents plugin. Moves the decoupled boot-time
assembly helpers into registry.ts: loadCodeAgents, hasCodeAgentSources (now
internal), resolveDefaultAgent, and the AgentSource type. buildAgentRegistry
stays as the orchestrator that wires them. Also merges a duplicate import in
skill-loader.ts.

agents.ts: 2152 -> 2088 lines. typecheck + 394 agent tests green.

Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>

…tch.ts

Step 4 (final) of splitting the agents plugin. Moves dispatchToolCall +
runSubAgent — the tool-call budget, approval gate, and sub-agent recursion —
into tool-dispatch.ts as free functions over RunState + a ToolDispatchDeps
object. The plugin builds deps via toolDispatchDeps(); the two executeTool
closures call the free function. RunState moves with them. Tests updated to
invoke the free functions (deps built from the plugin's own builder).

agents.ts: 2088 -> 1839 lines (2512 -> 1839 across all four steps). typecheck +
394 agent tests green.

Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>

…g.ts

Moves resolvedApprovalPolicy / resolvedLimits defaulting into pure functions
over AgentsPluginConfig. The getters keep the approval-policy memo cache and
delegate. Config-only interface; both now unit-testable in isolation.

agents.ts: 1839 -> 1812 lines. typecheck + agent tests green.

Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>

Moves the active-stream map + per-user counter (the O(1) concurrency-limit
check) into a StreamRegistry class. The plugin holds one instance and keeps
trackStream/untrackStream/countUserStreams as delegators; cancel/approve read
via streams.get(). Tests inject via trackStream and assert via the registry.

agents.ts: 1812 -> 1784 lines. typecheck + 394 agent tests green.

Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>

The previous commit named it StreamRegistry, colliding with the existing
SSE-layer StreamRegistry in src/stream/ (connection/event-buffer tracking used
by StreamManager). They're different concepts; renamed the agents-plugin one to
ActiveStreamTracker (tracks active streams + per-user counts for the O(1)
concurrency limit) to avoid the name clash.

Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>

…ing)

The prior commit's git add hit the already-deleted stream-registry.ts path,
aborted, and recorded only the deletion — leaving the pushed tip non-compiling
(agents.ts imported the removed file; active-stream-tracker.ts was uncommitted).
This adds the new module and the agents.ts import/usage so the tree builds.

Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>